Skip to content

Conversation

rffontenelle
Copy link
Collaborator

Aplica powrap a todos os arquivos .po do repositório e adiciona execução de powrap logo após baixar traduções de tx pull toda vez que make pull é executado.

A opção --modified faz com que powrap identifique quais arquivos .po foram alterados ou são novos, e atua apenas neles.

@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

@rffontenelle demorei pra revisar e agora estamos com conflitos. Sabe a diferença de tempo rodando o powrap sem o --modified? As vezes nem precisa do argumento.

@rffontenelle
Copy link
Collaborator Author

@rougeth O tempo de execução sem --modified não é nada proibitivo. Achei interessante usá-lo para o script não "tocar" arquivos não envolvidos no commit, mas talvez não haja problema aplicar a todos os arquivos substituindo a linha
$(VENV)/bin/powrap --modified --quiet
por
$(VENV)/bin/powrap --quiet *.po **/*.po

Prefere assim?

@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

Teoricamente, ele não deveria alterar nenhum arquivo que não houve alteração, né? Acho que podemos testar, o que acha?

@rffontenelle
Copy link
Collaborator Author

Bora. Mandei um commit atualizando conforme últimos commits no 3.8, e sem alterações minhas aos arquivos po. Uma vez mesclado, deve aplicar powrap a todos os arquivos do repositório.

@rougeth rougeth merged commit 1abe5c6 into python:3.8 Jan 28, 2020
@rougeth
Copy link
Collaborator

rougeth commented Jan 28, 2020

Show!

@rffontenelle rffontenelle deleted the add-powrap branch January 28, 2020 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants